Skip to content

Update logo#34

Merged
melanke merged 5 commits intoBuidlGuidl:mainfrom
Mrwicks00:update-logo
May 29, 2025
Merged

Update logo#34
melanke merged 5 commits intoBuidlGuidl:mainfrom
Mrwicks00:update-logo

Conversation

@Mrwicks00
Copy link
Contributor

@Mrwicks00 Mrwicks00 commented May 14, 2025

Description

This PR implements the Batch 16 main page updates by adding a new logo, replacing the favicon, setting the page title, and adding a subtitle, as per the task requirements (issue #7 ). The changes enhance the batch page's branding and ensure visual consistency.

Key Changes:

  • Added a new batch logo (batch-logo.png) to the main content area of the Batch 16 page at packages/nextjs/app/page.tsx, designed from the DALL-E logo featuring a stylized Ethereum diamond symbol with the number '16' in a blue-to-purple gradient, placed in public/batch-logo.png.
  • Replaced the existing favicon with a new favicon.ico (matching the batch logo), placed in public/favicon.ico, and removed the old favicon.png.
  • Fixed favicon loading issue by updating the getMetadata function in utils/scaffold-eth/getMetadata.ts to set icons.icon to /favicon.ico, resolving browser requests for the old favicon.png (404 errors).
  • Converted batch-logo.png (1.5 MB) to batch-logo.webp, reducing the size to ~115 KB.
  • Updated references in app/batch-16/page.tsx to use batch-logo.webp.
  • Fixed <Image> console error by removing fill and adding width={288} and height={288}.
  • Forced favicon update by renaming favicon.ico to favicon-v2.ico to bypass browser caching.
  • Removed unrelated files (page.tsx and 596.avif) from the PR.
  • @melanke I confirmed there were no changes to package.json in this branch—it might have been a misunderstanding. I also reverted the favicon renaming from favicon-v2.ico back to favicon.ico as part of cleaning up.

Screenshots:

![Batch 16 Page ]image

Additional Information

Your ENS/address:

  • Address: 0xB24023434c3670E100068C925A87fE8F500d909a
  • ENS: None

@vercel
Copy link

vercel bot commented May 14, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
batch16.buidlguidl.com ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 29, 2025 0:35am

Copy link
Collaborator

@melanke melanke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for this cool logo!

Could you also update this part on the header?
284656861-52a77020-0c08-4136-8cc5-244bc3bd088a

Please replace the image and both texts. Thanks.

Oh and please, link the issue number to the PR description.

And here is my review:

@phipsae phipsae mentioned this pull request May 20, 2025
2 tasks
@phipsae
Copy link
Contributor

phipsae commented May 22, 2025

@Mrwicks00 just a quick reminder that we wait for your changes

Copy link
Contributor

@phipsae phipsae left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thx! Just a tiny little change and then we merge!

@melanke melanke merged commit b7e848b into BuidlGuidl:main May 29, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants